Closed in favor of #3724

All Submissions:

• Have you followed the Newspack Contributing guideline?
• Does your code follow the WordPress' coding standards and VIP Go coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

This PR changes the logic of the guest author migration script.

Basically, the script was checking if a guest user already existed as a regular WP user based on the user_login. I'm changing that to check for the user_nicename.

In a regular situation, the nicename will usually be the same thing as the login, but there are cases where they can be different. One of them is when you migrated users from another system and treated these fields in a different way in the database.

CAP uses the $user->user_nicename to create and match the author term, and that's why this is the field we should look at.

How to test the changes in this Pull Request:

1. Create a few guest authors and assign posts to them.
2. Run the script and make sure the guest contributors users are created and the posts are correctly assigned.
3. Create a guest author and make sure there is a wp user that has its nicename with the same value of the GA's login
4. Run the script and make sure no new WP user is created, and the posts that were assigned to that GA is assigned to this WP user

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?